Skip to content

Conversation

@nscuro
Copy link
Member

@nscuro nscuro commented Jun 15, 2024

  • Only build against Temurin, drop Zulu
  • Build against Java 8, 11, 17, and 21, instead of just 8
  • Pin 3rd party actions to their digest
  • Drop permissions on workflow-level, define them on job-level
  • Update JamesIves/github-pages-deploy-action to the latest version
  • Update release-drafter/release-drafter to the latest version
  • Run the Maven CI workflow only for pushes to, or pull request into, master. Previously we had duplicate runs for PRs.
  • Reduce build output noise by disabling Maven's transfer progress logs
  • Record test coverage with JaCoCo
  • Publish coverage reports to Codacy
    • Because we build against multiple Java versions, we'll upload only the report generated from the latest Java build

* Only build against Temurin, drop Zulu
* Build against Java 8, 11, 17, and 21, instead of just 8
* Pin 3rd party actions to their digest
* Drop `permissions` on workflow-level, define them on job-level
* Update `JamesIves/github-pages-deploy-action` to the latest version
* Update `release-drafter/release-drafter` to the latest version
* Run the `Maven CI` workflow only for pushes to, or pull request into, `master`. Previously we had duplicate runs for PRs.
* Reduce build output noise by disabling Maven's transfer progress logs
* Record test coverage with JaCoCo
* Publish coverage reports to Codacy
  * Because we build against multiple Java versions, we'll upload only the report generated from the latest Java build

Signed-off-by: nscuro <[email protected]>
@nscuro nscuro force-pushed the improve-ci-pipeline branch from 0cf2b40 to c21d53e Compare June 15, 2024 12:31
Copy link
Contributor

@mr-zepol mr-zepol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 LGTM

@nscuro nscuro merged commit a9ca98e into master Jun 18, 2024
@nscuro nscuro deleted the improve-ci-pipeline branch June 18, 2024 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants